Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update BancoSicredi.CNAB240.cs #335

Merged
merged 2 commits into from
Dec 18, 2024
Merged

Conversation

brunogoulart28
Copy link
Contributor

#334 - Ajustado identificação do titulo da empresa na remessa do sicred

BoletoNet#334 - Ajustado identificação do titulo da empresa na remessa do sicred
@carloscds
Copy link
Collaborator

@brunogoulart28 Sua alteracao não esta correta, o campo utilizado é o NumeroDocumento, este é que você precisa preencher na classe boleto.

@marcianobandeira
Copy link
Contributor

Será que isso não vai quebrar aplicações em andamento? Talvez, se realmente for necessário, o ideal seria criar uma configuração para manter o funcionando atual por padrão. Ou se o comportamento atual realmente estiver errado, seria melhor documentar o embasamento para não restar duvidas

@carloscds
Copy link
Collaborator

@brunogoulart28 Continua incorreto. veja o comentário acima.

@jader1313
Copy link

@carloscds entendo que você esteja confundindo as coisas. O campo citado, posição 196 a 220, não é o campo para o número do documento, mas sim para um código interno. A descrição para a posição indica:
// 196 a 220 - Identificação do título na empresa.
Peguei o código de mais um banco(Banco do Brasil) para confirmar e também é utilizado o NumeroControleParticipante:
BoletoNet/BoletoNetCore/tree/master/BoletoNetCore/Banco/BancoBrasil.CNAB240.cs
linha 315: reg.Adicionar(TTiposDadoEDI.ediAlphaAliEsquerda_____, 0196, 025, 0, boleto.NumeroControleParticipante, ' ');

@jader1313
Copy link

O NumeroDocumento vai na posição 63 tanto no Sicredi como no Banco Brasil:
reg.Adicionar(TTiposDadoEDI.ediAlphaAliEsquerda_____, 0063, 015, 0, boleto.NumeroDocumento, ' '); // 063 a 077 - Nº do documento de cobrança

@carloscds
Copy link
Collaborator

@carloscds entendo que você esteja confundindo as coisas. O campo citado, posição 196 a 220, não é o campo para o número do documento, mas sim para um código interno. A descrição para a posição indica: // 196 a 220 - Identificação do título na empresa. Peguei o código de mais um banco(Banco do Brasil) para confirmar e também é utilizado o NumeroControleParticipante: BoletoNet/BoletoNetCore/tree/master/BoletoNetCore/Banco/BancoBrasil.CNAB240.cs linha 315: reg.Adicionar(TTiposDadoEDI.ediAlphaAliEsquerda_____, 0196, 025, 0, boleto.NumeroControleParticipante, ' ');

Identificacao do Documento na Empresa é o seu numero do sistema, como você controla o seu financeiro. Ele vem depois no retorno para que você possa, eventualmente, usar para baixa.

@jader1313
Copy link

seu numero do sistema, como você controla o seu financeiro. Ele vem depois no retorno para que você possa, eventualmente, usar para baixa.

Exatamente isso, mas no caso do SICREDI está retornando o NumeroDocumento pois é o que está sendo colocado na remessa na posição 196 ao invés de colocar o NumeroControleParticipante. Essa foi a correção implementada no PR do Bruno.

@carloscds
Copy link
Collaborator

seu numero do sistema, como você controla o seu financeiro. Ele vem depois no retorno para que você possa, eventualmente, usar para baixa.

Exatamente isso, mas no caso do SICREDI está retornando o NumeroDocumento pois é o que está sendo colocado na remessa na posição 196 ao invés de colocar o NumeroControleParticipante. Essa foi a correção implementada no PR do Bruno.

Ok, Mas a mudança é na remessa e nao no retorno. A Remessa é o numero do documento, nao ?

@jader1313
Copy link

seu numero do sistema, como você controla o seu financeiro. Ele vem depois no retorno para que você possa, eventualmente, usar para baixa.

Exatamente isso, mas no caso do SICREDI está retornando o NumeroDocumento pois é o que está sendo colocado na remessa na posição 196 ao invés de colocar o NumeroControleParticipante. Essa foi a correção implementada no PR do Bruno.

Ok, Mas a mudança é na remessa e nao no retorno. A Remessa é o numero do documento, nao ?

Nós precisamos enviar o NumeroControleParticipante na posição 196 do arquivo de REMESSA, para que o banco retorne esse mesmo número no arquivo de RETORNO.
O NumeroDocumento já é informado na posição 63 da remessa, não tem porque repetí-lo na posição 196.

@jader1313
Copy link

A maioria dos bancos já está dessa forma, não sei se todos, mas a implementação para o SICREDI está incorreta e diferente dos outros bancos.

@carloscds carloscds merged commit 3f17cfa into BoletoNet:master Dec 18, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants